Skip to content

fix: use spec Error.cause property instead of legacy cause() function - #112

Draft
mason-albert wants to merge 1 commit into
masterfrom
masona/nerror-v2
Draft

fix: use spec Error.cause property instead of legacy cause() function#112
mason-albert wants to merge 1 commit into
masterfrom
masona/nerror-v2

Conversation

@mason-albert

Copy link
Copy Markdown

@netflix/nerror@2.0.0-rc.0 replaces the non-standard cause() method with the native Error.cause property. Update HttpError.toJSON() and the serializer's cause-chain walk accordingly, and fix a latent bug where toJSON() was serializing the cause function reference itself rather than a value.

@netflix/nerror@2.0.0-rc.0 replaces the non-standard cause() method with
the native Error.cause property. Update HttpError.toJSON() and the
serializer's cause-chain walk accordingly, and fix a latent bug where
toJSON() was serializing the cause function reference itself rather
than a value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant